Adapt Button to styled-react and remove sx #6904
Conversation
🦋 Changeset detectedLatest commit: bd84452 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
There was a problem hiding this comment.
Pull Request Overview
This PR adapts the Button component to the styled-react package and removes the sx prop implementation from the main @primer/react package. The changes introduce custom styled-react implementations for Button, IconButton, and ActionMenu components that support the SxProp interface for backward compatibility.
Key changes:
- Creates new styled-react wrapper components for Button, ActionMenu, and IconButton
- Moves CSSCustomProperties type to a shared location in sx.ts
- Updates package exports to use the new styled-react implementations
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/styled-react/src/sx.ts | Adds CSSCustomProperties type export for shared use |
| packages/styled-react/src/index.tsx | Updates exports to use new styled-react components instead of direct @primer/react exports |
| packages/styled-react/src/components/PageHeader.tsx | Updates import to use shared CSSCustomProperties type |
| packages/styled-react/src/components/IconButton.tsx | Creates new styled-react wrapper for IconButton with sx prop support |
| packages/styled-react/src/components/Button.tsx | Creates new styled-react wrapper for Button with custom sx prop handling |
| packages/styled-react/src/components/ActionMenu.tsx | Creates new styled-react wrapper for ActionMenu with sx prop support |
|
😢 Hi from github/github-ui. The integration workflow has failed: https://github.com/github/github-ui/actions/runs/17979041956 |
github-actions
bot
added
the
integration-tests: recommended
|
👋 Hi, this pull request contains changes to the source code that github/github depends on. If you are GitHub staff, we recommend testing these changes with github/github using the integration workflow. Thanks! |
pksjce
had a problem deploying
to
github-pages
GitHub Actions
Failure
github-actions
bot
requested a deployment
to
storybook-preview-6904
github-actions
bot
requested a deployment
to
storybook-preview-6904
Working on #6760
Changelog
New
Changed
Removed
Rollout strategy
Testing & Reviewing
Merge checklist